Skip to content

Removed result set + ser/de refactor#13

Merged
YaroslavLitvinov merged 8 commits into
mainfrom
yaro/clean_code_and_refactor_result_set
Nov 14, 2025
Merged

Removed result set + ser/de refactor#13
YaroslavLitvinov merged 8 commits into
mainfrom
yaro/clean_code_and_refactor_result_set

Conversation

@YaroslavLitvinov
Copy link
Copy Markdown
Contributor

@YaroslavLitvinov YaroslavLitvinov commented Nov 13, 2025

  • Removed ResultSet
  • Removed excessive serdes
  • Removed id field from QueryResult
  • Remove test test_bug_1662_duplicate_columns_names as not related from Executor as it is pure ser/de bug

Copy link
Copy Markdown
Contributor

@DanCodedThis DanCodedThis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@YaroslavLitvinov YaroslavLitvinov merged commit d23b138 into main Nov 14, 2025
3 checks passed
@YaroslavLitvinov YaroslavLitvinov deleted the yaro/clean_code_and_refactor_result_set branch November 14, 2025 18:56
Vedin pushed a commit that referenced this pull request Dec 2, 2025
* # This is a combination of 14 commits.
# This is the 1st commit message:

Upload CSV to table

# This is the commit message #2:

Upload CSV to table

# This is the commit message #3:

Fix linter

# This is the commit message #4:

Remove unused error

# This is the commit message #5:

Fix clippy

# This is the commit message #6:

Fix clippy

# This is the commit message #7:

Fix linter

# This is the commit message #8:

Fix clippy

# This is the commit message #9:

Fix clippy

# This is the commit message #10:

Fix clippy

# This is the commit message #11:

Fix fmt

# This is the commit message #12:

Add pre-commit

# This is the commit message #13:

Parse merge into

# This is the commit message #14:

Parse merge into

* Merge

* Add full message text into the error

* Fix

* Update MERGE to insert only not matched records

* Update deps

* Update MERGE to insert only not matched records
Vedin pushed a commit that referenced this pull request Dec 2, 2025
* # This is a combination of 14 commits.
# This is the 1st commit message:

Upload CSV to table

# This is the commit message #2:

Upload CSV to table

# This is the commit message #3:

Fix linter

# This is the commit message #4:

Remove unused error

# This is the commit message #5:

Fix clippy

# This is the commit message #6:

Fix clippy

# This is the commit message #7:

Fix linter

# This is the commit message #8:

Fix clippy

# This is the commit message #9:

Fix clippy

# This is the commit message #10:

Fix clippy

# This is the commit message #11:

Fix fmt

# This is the commit message #12:

Add pre-commit

# This is the commit message #13:

Parse merge into

# This is the commit message #14:

Parse merge into

* Merge

* Add full message text into the error

* Tmp

* Fix

* Implement qualify

* Update deps

* Update MERGE to insert only not matched records

* Update deps

* Update MERGE to insert only not matched records

* Add lt lteq
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants